Skip to content

Conversation

@tosuke
Copy link
Contributor

@tosuke tosuke commented Oct 26, 2025

供養

戦略

  • PR 上で plan を生成し artifact に (ロックがないので refresh しない)
  • Merge Queue の並列数を1に制限した上で apply を実行する
  • apply に成功したら main ブランチにマージ、そうでなければ plan を再計算して PR に戻す

懸念

  • シークレット管理
    • plan のためには state の読み取り権限が必要だが、公開リポジトリでこれは難しい
  • 管理コスト
    • それなりに複雑なワークフローが必要になる

戦略
- PR 上で plan を生成し artifact に (ロックがないので refresh しない)
- Merge Queue の並列数を1に制限した上で apply を実行する
- apply に成功したら main ブランチにマージ、そうでなければ plan を再計算して PR に戻す

懸念
- シークレット管理
    - plan のためには state の読み取り権限が必要だが、公開リポジトリでこれは難しい
- 管理コスト
    - それなりに複雑なワークフローが必要になる
changes: ${{ steps.filter.outputs.changes }}
steps:
- uses: actions/[email protected]
- uses: dorny/[email protected]

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium

Unpinned 3rd party Action 'Terraform' step
Uses Step: filter
uses 'dorny/paths-filter' with ref 'v3.0.2', not a pinned commit hash
@tosuke
Copy link
Contributor Author

tosuke commented Oct 27, 2025

changes に引っ掛かったら environment を使うようにするという手もある

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants